Skip to content

Conversation

@kevin-kortum-trustedshops

This PR add the possibility to override the sonar scanner cli version for users of the orb and fixes minor warnings by shellcheck regarding missing quotations to prevent unwanted variable expansions.

…the default sonar scann cli version. also fixes quoting warnings by shellcheck

Signed-off-by: Kevin Kortum <[email protected]>
@kevin-kortum-trustedshops
Copy link
Author

@antoine-vinot-sonarsource @SamirM-BE Would love to get a review from you for this :)

@antoine-vinot-sonarsource
Copy link
Contributor

Hi @kevin-kortum-trustedshops,
Thank you very much for your contribution. 🚀
The change makes sense, it's consistent with what we do in other products.
Please bear with us, It would require a bit more work on our side: adding test, documentation update and release. And it's not in our priorities at the moment.
I'll try to come back to it asap.

@kevin-kortum-trustedshops
Copy link
Author

Hi @antoine-vinot-sonarsource
i have seen some change upstream, which affect the version number. Feel free to update my branch in case you want to merge it or let me know if you need me to rebase. Also, if i can help with updating any documentation, let me know what i would need to edit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants